Skip to content

fix(backend): remove dead Crisp unread endpoint - #11995

Merged
kodjima33 merged 1 commit into
BasedHardware:mainfrom
shubhxho:codex/fix-11429-crisp-unread
Aug 22, 2026
Merged

kodjima33 merged 1 commit into
BasedHardware:mainfrom
shubhxho:codex/fix-11429-crisp-unread

Conversation

@shubhxho

@shubhxho shubhxho commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Closes #11429.

What changed and why

Remove the retired GET /v1/crisp/unread implementation while preserving the live screen-activity route on the same router. Remove its route-policy entry and record the intentional API removal in the app-client compatibility checker.

Product invariants affected

none

How it was verified

  • PYTHONPATH=backend pytest -q backend/tests/unit/test_desktop_screen_crisp.py — 10 passed, including a route-level assertion that /v1/crisp/unread is now absent while the screen-activity suite remains green.
  • make preflight — repository PR contracts passed.

Failure class (fixes)

Failure-Class: none

Review in cubic

Desktop Crisp client is gone; keep /v1/screen-activity/sync on the same
router. Record the route in DELIBERATELY_REMOVED_ENDPOINTS.

Refs: BasedHardware#11429
Failure-Class: none
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@kodjima33 kodjima33 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Backend fix removing dead endpoint, confidence 5/5: closes #11429, scoped diff, regression test added, confirmed endpoint still present on main so not redundant.

@kodjima33
kodjima33 merged commit b5ad95e into BasedHardware:main Aug 22, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Backend: /v1/crisp/unread has no callers after the desktop Crisp removal

2 participants